Skip to content
This repository was archived by the owner on Aug 5, 2025. It is now read-only.

fix: resolve Python Datetime warnings #173

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

fix: resolve Python Datetime warnings #173

wants to merge 1 commit into from

Conversation

emmanuel-ferdman
Copy link

Summary

This small PR resolves the datetime library warnings:

DeprecationWarning: datetime.datetime.utcnow() is deprecated and scheduled for removal in a future version. Use timezone-aware objects to represent datetimes in UTC: datetime.datetime.now(datetime.UTC). or datetime.datetime.utcnow()

DeprecationWarning: datetime.datetime.utcfromtimestamp() is deprecated and scheduled for removal in a future version. Use timezone-aware objects to represent datetimes in UTC: datetime.datetime.fromtimestamp(timestamp, datetime.UTC).

Note that .replace(tzinfo=None) allows to keep the original behavior where the time appears as a naive UTC timestamp (i.e., without any timezone offset).

@emmanuel-ferdman
Resolve Python Datetime warnings
720c59b 
Signed-off-by: Emmanuel Ferdman <[email protected]>
@emmanuel-ferdman emmanuel-ferdman mentioned this pull request May 15, 2025
Merged
@emmanuel-ferdman
Copy link
Author

@willydouhard could you please review this small PR when you get the chance? thanks 😄

@hayescode
Copy link

@emmanuel-ferdman I'm just going to move the datetime helpers from literalai to chainlit and handle this all there.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@emmanuel-ferdman @hayescode